Skip to content

Conversation

@itsatan
Copy link
Contributor

@itsatan itsatan commented Apr 22, 2023

  1. 新增了copy功能
  2. 移除了不必要的yarn.lock文件
  3. 增加了size-limit容量(emmm...10KB 不够用了)

如果可以合入辛苦改一下example图片和README

Copy link
Owner

@jianxiaoBai jianxiaoBai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个功能看起来很挺实用的,感谢贡献,这个 review 能完善一下的话我就 merge 了。

if (Array.isArray(sourceData)) {
sourceData.splice(+key, 0, value)
} else {
Reflect.set(sourceData, key + '副本', Reflect.get(sourceData, key))
Copy link
Owner

@jianxiaoBai jianxiaoBai Apr 24, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. 这里把 "副本" 改成 "copy" 吧
  2. 这还有个小问题就是只能新增一个,不能连续新增,新增时 key 的副本也会进行自增,如 key copy 1/ key copy

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

由于field文本框宽度为100px,此处修改为key copy 1 - key copy n 会造成key过长不方便阅读,所以我改为了key-n这种类型,去掉了copy,并且复制出来的数据禁止了的copy功能

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants